12540ca8000fcebe4eb2c19cc2afa245a131b8aa,authz/authz-tool/tool/src/java/org/sakaiproject/authz/tool/RealmsAction.java,RealmsAction,readRoleForm,#RunData#SessionState#,1145

Before Change


		role.setDescription(StringUtils.trimToNull(data.getParameters().getString("description")));

		// providerOnly
		String providerOnlyString = (StringUtils.trimToNull(data.getParameters().getString("providerOnly")));
		role.setProviderOnly("true".equals(providerOnlyString));		

		// for each lock set, give it to the role

After Change


		role.disallowAll();

		String descriptionString = StringUtils.trimToNull(data.getParameters().getString("description"));
		String providerOnlyString = StringUtils.trimToNull(data.getParameters().getString("providerOnly"));
		
		//Role can't be null at this point
		if (descriptionString == null) {
			descriptionString = role.getDescription();
		}
		
		if (providerOnlyString == null) {
			providerOnlyString = String.valueOf(role.isProviderOnly());
		}